Skip to content

Conversation

@sanket1729
Copy link
Member

@sanket1729 sanket1729 commented Nov 5, 2022

Earlier, we only implemented `expr_raw_pkh` which was used as shortcut
for c:expr_raw_pk_h. Note the inclusion of additional _
psbts where we needed to dissatisfy pkh did not work properly as we did
not implement the mapping pkh->pk required for dissatisfaction of pkh
fragment. We now look for this mapping in bip32 derivation field of
psbt.

Our update_utxo_from_descriptor API correctly procesess these fields.
Same example as posted in discussions
@apoelstra
Copy link
Member

I'm not sure that we can change the Display output in a patch release.

@apoelstra
Copy link
Member

Changing the signature of Satisfier::lookup_raw_pkh_pk we definitely can't do in a patch release.

@apoelstra
Copy link
Member

All changes in 7e1a7a0 look good to me, but they're all API-breaking and I think we need to release 9.0.0 with them.

@sanket1729
Copy link
Member Author

sanket1729 commented Nov 7, 2022 via email

@evd0kim
Copy link

evd0kim commented Nov 10, 2022

@apoelstra what is expected date for the next release if the fix goes with it?

@sanket1729
Copy link
Member Author

@engenegr, you can expect a new release early next week.

@sanket1729 sanket1729 changed the title Release 8.0.1 Release 9.0.0 Nov 12, 2022
@sanket1729
Copy link
Member Author

Changed to release 9.0.0

@apoelstra
Copy link
Member

@sanket1729 you did not change the actual commit to 9.0.0!

@sanket1729
Copy link
Member Author

sanket1729 commented Nov 14, 2022

@apoelstra , lol. Good catch. I think you mean Cargo.toml file?

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK a113e35

@apoelstra apoelstra merged commit a7bfff4 into rust-bitcoin:master Nov 14, 2022
@apoelstra
Copy link
Member

Tagged & published

heap-coder added a commit to heap-coder/rust-miniscript that referenced this pull request Sep 27, 2025
a113e35923dc094b2554209d2042ac8d7683ebb7 Release 9.0.0 (sanket1729)
63ceb5b38d641c543f01f955728627ea2a89444a Add psbt example for sign and finalize (sanket1729)
8f869926714c5c7273b217bd3f72d5cab26d6560 Don't run rawpkh logic through pkh (sanket1729)
815fd1c9b368f800bb67d1ecdd49c533394d4b8c Bug fix: pkh->pk lookup API (sanket1729)
43abc43bb0b83ca9da61f80ed3cc848e4e34d762 Use expr_raw_pk_h for Terminal::RawPkH (sanket1729)

Pull request description:

  - Cleanup some pkh code in satisfaction
  - Fixes #483
  - Fixes another bug dealing with dissatisfaction of thresh inside pkh
  - Adds example test vector from discussions
  - release 9.0.0 with above fixes

ACKs for top commit:
  apoelstra:
    ACK a113e35923dc094b2554209d2042ac8d7683ebb7

Tree-SHA512: 373d80c5bc03032635ce0dfe6426b17970f1590012482ad3cc76b37337fe2f53e51b32e1fe11c449ec38bf4d4a0ba62d38f7316a148773e2b1854b8f9f1a877c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

We should not be running pkh logic through rawpkh

3 participants